-
Notifications
You must be signed in to change notification settings - Fork 303
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update capnp #2023
update capnp #2023
Conversation
@@ -39,10 +39,10 @@ apple_support_dependencies() | |||
|
|||
http_archive( | |||
name = "capnp-cpp", | |||
sha256 = "21ae72a199a995b72e9bd359d4815539158d93a15cf36e284ef201fde7338c3c", | |||
strip_prefix = "capnproto-capnproto-8ba7a6e/c++", | |||
integrity = "sha256-I/jF25bRniNfmXR4KdeXTnwetKPKEm6fkYfibpZFYoU=", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do I generate this new format?
The old format, I would just do: curl -L $URL | sha256sum
But off the top of my head I'm not sure how I'd generate this base64.
Can we maybe just stick with the old format? Or is there some meaningful advantage to this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do I generate this new format?
you just run bazel without any integrity attribute and it tells you what to add (previously it was suggesting sha256
thats why we have it everywhere):
INFO: Invocation ID: a7c49738-0fe1-47d0-b3aa-2ea74014898c
DEBUG: Rule 'capnp-cpp' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-I/jF25bRniNfmXR4KdeXTnwetKPKEm6fkYfibpZFYoU="
DEBUG: Repository capnp-cpp instantiated at:
/home/maizatskyi/src/github.com/cloudflare/workerd/WORKSPACE:40:13: in <toplevel>
The old format, I would just do: curl -L $URL | sha256sum
I have never actually downloaded any archive and was always relying on bazel to tell me :)
Can we maybe just stick with the old format?
I personally don't care. Up to you.
Or is there some meaningful advantage to this change?
This is a "standard" thing: https://developer.mozilla.org/en-US/docs/Web/Security/Subresource_Integrity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh huh, I guess I never tried running bazel without fully updating the hash first.
No description provided.